Skip to content

Fix update interval edge case, remove duplicate code and add better startup warning#134

Closed
anirudhgupta109 wants to merge 1 commit into
ChrisTitusTech:mainfrom
anirudhgupta109:update-interval-fixes
Closed

Fix update interval edge case, remove duplicate code and add better startup warning#134
anirudhgupta109 wants to merge 1 commit into
ChrisTitusTech:mainfrom
anirudhgupta109:update-interval-fixes

Conversation

@anirudhgupta109

Copy link
Copy Markdown
Contributor

No description provided.

…tual info to warning

In the previous implementation (PR ChrisTitusTech#127), there was an edge case where a powershell update check will take place and a profile update check might not
due to floating point vs int values (.Date vs without (L84 vs L125 in aforementioned PR)).
Commonize the checks since no situation exists where one runs but the other does not.

Add better dev-centered warning message since the previous warning was made to address the fact the profile has transitioned from update checks every launch
to a periodic check, now that users are aware of it, we can notify them when the next update will occur rather than the fact it was just skipped. (Issue ChrisTitusTech#130)

Signed-off-by: Anirudh Gupta <anirudhgupta109@gmail.com>
@anirudhgupta109 anirudhgupta109 deleted the update-interval-fixes branch April 17, 2025 05:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants